-
Notifications
You must be signed in to change notification settings - Fork 38.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Delete all files and all hidden files without . & .. error for PV recycler #16723
Delete all files and all hidden files without . & .. error for PV recycler #16723
Conversation
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
1 similar comment
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
Labelling this PR as size/XS |
@wulonghui thanks. I am not a bash expert :) LGTM |
@k8s-bot ok to test
|
GCE e2e test build/test passed for commit 86abfb8. |
The author of this PR is not in the whitelist for merge, can one of the admins add the 'ok-to-merge' label? |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e test build/test passed for commit 86abfb8. |
Automatic merge from submit-queue |
Auto commit by PR queue bot
Can this fix be merged/back-ported to v1.1.x? |
backport PR is in flight |
…3-upstream-release-1.1 Automated cherry pick of #16723 upstream release 1.1
Unfortunately the fix didn't work
It looks like the "find" command from gcr's busybox does not support -delete option. |
same as @tanen01 . the CLI for find is wrong in this change |
…k-of-#16723-upstream-release-1.1 Automated cherry pick of kubernetes#16723 upstream release 1.1
…k-of-#16723-upstream-release-1.1 Automated cherry pick of kubernetes#16723 upstream release 1.1
When PV recycling, it failed:
The default PV recycler Pod command to delete all files and all hidden files is:
this will cause error, because it try to delete . and ..
so replace the rm command by: